The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 14, 2026 1:14am

PR Summary

Low Risk
Low risk UI navigation tweak: replaces Next.js route replacement with window.history.replaceState, which could affect expected router-driven state updates but does not touch data or auth logic.

Overview
Prevents a brief UI flash when a new chat session is created by stopping the Next.js router.replace transition.

When the first chat session event assigns a new chatId, the hook now updates the URL via window.history.replaceState to /workspace/${workspaceId}/task/${chatId} instead of triggering a client-side route change.

^{Reviewed by Cursor Bugbot for commit 5ae0b42. Bugbot is set up for automated code reviews on this repo. Configure here.}

Greptile Summary

Replaces router.replace / router.push with window.history.replaceState when assigning a new chat ID to the URL, bypassing Next.js's routing lifecycle and eliminating the visual flash between the home page and the new-chat view.

The router variable (line 712) and the useRouter import (line 4) are now unused dead code: router is only referenced in the useCallback dependency array (line 2174) but never called inside the callback body.

Confidence Score: 5/5

Safe to merge — the fix is correct and the only remaining finding is dead-code cleanup (P2).

The core change (using window.history.replaceState instead of router navigation) correctly eliminates the flash with no functional regression. The only open finding is an unused router variable/import left over from the refactor, which does not affect runtime behavior.

apps/sim/app/workspace/[workspaceId]/home/hooks/use-chat.ts — minor cleanup of unused router/useRouter.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant HomeChat as useChat (home)
    participant API as Mothership API
    participant BrowserHistory as window.history

    User->>HomeChat: sendMessage()
    HomeChat->>API: POST /api/mothership/chat
    API-->>HomeChat: SSE session event (chatId assigned)
    HomeChat->>BrowserHistory: replaceState(null, '', /workspace/{id}/task/{chatId})
    Note over HomeChat,BrowserHistory: URL silently updated — no Next.js route change, no flash
    API-->>HomeChat: SSE text/tool events
    HomeChat->>HomeChat: setMessages(), setResolvedChatId()

_{Reviews (1): Last reviewed commit: "fix(ui): fix flash between home and new ..." | Re-trigger Greptile}